Skip to content

[TSK-73] 어드민 공지 응답, 리뷰 조회 응답 updatedAt, createdAt 추가#314

Merged
boyekim merged 2 commits intomainfrom
boyekim/TSK-73
Apr 16, 2026
Merged

[TSK-73] 어드민 공지 응답, 리뷰 조회 응답 updatedAt, createdAt 추가#314
boyekim merged 2 commits intomainfrom
boyekim/TSK-73

Conversation

@boyekim
Copy link
Copy Markdown
Member

@boyekim boyekim commented Apr 12, 2026

작업 내용

  • 어드민 공지 수정 후 응답과 어드민 공지 조회 시 응답에 updatedAt과 createdAt을 둘 다 반환하도록 추가했습니다.

  • 리뷰 조회 시 createdAt 응답을 추가해주었습니다.

고민 지점과 리뷰 포인트

  • 제가 혹시 잘못된 곳에 추가했는지 간단하게 같이 봐주시면 되겠습니다~!

@auto-assign auto-assign bot requested review from 2Jin1031 and goldm0ng April 12, 2026 07:46
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@auto-assign auto-assign bot requested a review from haeyoon1 April 12, 2026 07:46
@github-actions
Copy link
Copy Markdown

Test Results

209 tests   209 ✅  26s ⏱️
 49 suites    0 💤
 49 files      0 ❌

Results for commit 0da0222.

Copy link
Copy Markdown
Member

@haeyoon1 haeyoon1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

하나 코멘트 남겼습니당 👍

String detail,
OperationType operationType
OperationType operationType,
LocalDateTime createdAt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요기에 updatedAt 안써도 되나요?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제 생각엔 요기는 유저가 리뷰를 수정하는 기능을 제공하지 않기 때문에 불필요한 응답 필드라고 생각함니다~!
createdAt만 있어도 충분할 것 같다고 판단되는데 어떻게 생각하시나용?

Copy link
Copy Markdown
Member Author

@boyekim boyekim Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앗 맞아요~ 애초에 유저가 자기 리뷰를 볼 수 없고, 그래서 수정할 수도 없기 때문에 createdAt만 넣었습니다. 추후에 리뷰 확인 기능이 생기면 모를까 지금은 무의미할 것 같아서요.

Copy link
Copy Markdown
Member

@goldm0ng goldm0ng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

해윤님께서 남기신 코멘트에 의견 하나 남겼습니다!
그 외에는 적절히 잘 추가해주신 것아요 : )
수고하셨슴니다~!
머지해도 될 것 같아요 어프룹입니당

String detail,
OperationType operationType
OperationType operationType,
LocalDateTime createdAt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제 생각엔 요기는 유저가 리뷰를 수정하는 기능을 제공하지 않기 때문에 불필요한 응답 필드라고 생각함니다~!
createdAt만 있어도 충분할 것 같다고 판단되는데 어떻게 생각하시나용?

@boyekim boyekim merged commit e118bdd into main Apr 16, 2026
3 checks passed
@boyekim boyekim deleted the boyekim/TSK-73 branch April 16, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants